Repository navigation
chore(ci): run compliance harness 1.13.1 once per codec - #1026
Merged
Merged
Conversation
3 of 20 tasks
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 7, 2026 17:43
f2faf4d to
743d8ca
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-personless-option
branch
from
October 7, 2026 17:43
3f09f03 to
5e3fddc
Compare
Contributor
Author
|
@greptileai review |
eli-r-ph
marked this pull request as ready for review
October 7, 2026 20:44
dustinbyrne
approved these changes
Oct 8, 2026
dustinbyrne
left a comment
dustinbyrne
left a comment
Contributor
There was a problem hiding this comment.
The expanded compliance matrix and adapter forwarding look sound. Existing CI exercises both capture lanes across all three codecs. One small documentation follow-up: the standalone adapter-test setup needs the zstd extra.
AI-assisted review.
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 8, 2026 16:25
743d8ca to
ca9c26c
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-personless-option
branch
from
October 8, 2026 16:25
5e3fddc to
09f7144
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 8, 2026 18:13
ca9c26c to
e743d1e
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-personless-option
branch
from
October 8, 2026 18:13
09f7144 to
1355d44
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 8, 2026 21:12
e743d1e to
78973e7
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-personless-option
branch
2 times, most recently
from
October 8, 2026 22:54
316f808 to
b46dbfa
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
2 times, most recently
from
October 8, 2026 23:40
9dcf4df to
d31608d
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-personless-option
branch
2 times, most recently
from
October 10, 2026 23:35
233dc8a to
ed8a8d4
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
2 times, most recently
from
October 11, 2026 20:17
5c2b03e to
41336ea
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-personless-option
branch
from
October 11, 2026 20:17
ed8a8d4 to
0adfafd
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 14, 2026 03:11
41336ea to
ba13859
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-personless-option
branch
2 times, most recently
from
October 14, 2026 03:46
b25f805 to
608878e
Compare
The adapter now advertises capture_ai_v1 and event_options, passes options and uuids through unchanged, and takes its codec from COMPRESSION. The workflow runs the harness containers directly in a gzip/deflate/zstd matrix, so it no longer needs pull-requests: write.
The standalone adapter-test setup installs the zstd extra, matching CI.
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 14, 2026 03:46
ba13859 to
318305e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
💡 Motivation and Context
Stacked on #1025. The compliance harness skipped the AI lane and event options for this SDK, and its reusable workflow failed when posting a PR comment. This PR runs harness 1.13.1 with every capture suite the SDK supports, once per codec.
Changes
capture_ai_v1andevent_options, so both suites run./captureand/capture_aishare one handler that passesoptions,propertiesanduuidto the SDK unchanged.COMPRESSION(gzip, deflate or zstd; default gzip).contents: read.No SDK code changes.
💚 How did you test it?
🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Written with Cursor (Claude Opus) under the direction of the assignee.
🤖 Generated with Claude Code